inspector: Destroy popovers on unmap
authorCarlos Garnacho <carlosg@gnome.org>
Fri, 20 Feb 2015 11:46:52 +0000 (12:46 +0100)
committerCarlos Garnacho <carlosg@gnome.org>
Fri, 20 Feb 2015 13:44:17 +0000 (14:44 +0100)
As an implementation detail, the popover hooks the fade out animation
on ::hide. Destroying the popover right away here is not a problem, but
prevents the animation from actually running. ::unmap will be run after
the animation is finished, so destroy the popover there.

gtk/inspector/prop-list.c

index 75b3858a39748eeca5edb5b317a266da2e14288e..7a5aa72028840f80447ecb63d3d99e63cbea7ce4 100644 (file)
@@ -231,7 +231,7 @@ row_activated (GtkTreeView *tv,
 
   gtk_widget_show (popover);
 
-  g_signal_connect (popover, "hide", G_CALLBACK (gtk_widget_destroy), NULL);
+  g_signal_connect (popover, "unmap", G_CALLBACK (gtk_widget_destroy), NULL);
 
   g_free (name);
 }